Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove debug code from transpiled sources. #531

Merged
merged 2 commits into from
Oct 14, 2017
Merged

Conversation

autra
Copy link
Contributor

@autra autra commented Oct 13, 2017

Description

This contains 2 commits:

  • the first one correctly remove debug code from transpiled sources

  • the second is a basic refactoring

Motivation and Context

Before this patch, people using itowns as a module (as it should be used) had to manually define DEBUG in their own module bundler config, otherwise they had some undefined error. This commit correctly strips itowns out of these debug code during transpilation.

@autra autra closed this Oct 13, 2017
@autra autra changed the title Refactoring o Remove debug code from transpiled sources. Oct 13, 2017
@autra autra reopened this Oct 13, 2017
@autra
Copy link
Contributor Author

autra commented Oct 13, 2017

Referencing #529 here.

autra added 2 commits October 13, 2017 16:33
Before this patch, people using itowns as a module (as it should be) had
to manually define __DEBUG__ in their own module bundler config,
otherwise they had some undefined error. This commit correctly strips
itowns out of these debug code during transpilation.
@autra autra force-pushed the babel_strip_debug branch from 0ba87f2 to ff93713 Compare October 13, 2017 14:33
@autra autra requested a review from tbroyer October 13, 2017 14:38
@autra autra mentioned this pull request Oct 13, 2017
Copy link
Contributor

@tbroyer tbroyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@peppsac
Copy link
Contributor

peppsac commented Oct 14, 2017

Thanks @autra

@peppsac peppsac merged commit b56360d into master Oct 14, 2017
@peppsac peppsac deleted the babel_strip_debug branch October 14, 2017 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants